Skip to content

Modify test infrastructure and grunt tasks #744

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 5, 2015

Conversation

rosen-vladimirov
Copy link
Contributor

As common lib is now a separate npm package, it has its own tests, that must be run when executing grunt test or npm test in nativescript-cli. Modify test command in package.json in order to run the tests.
Change Gruntfile.js in order to ignore/build the correct typescript files in common lib.
Add Bluebird as dependency as it is required for common lib.
Update mocha version.
Remove chai and mocha definitions as they are part of common lib now.

@rosen-vladimirov rosen-vladimirov self-assigned this Aug 4, 2015
@rosen-vladimirov rosen-vladimirov added this to the 1.3.0 milestone Aug 4, 2015
@ns-bot
Copy link

ns-bot commented Aug 4, 2015

@rosen-vladimirov rosen-vladimirov force-pushed the vladimirov/unify-proton-and-master branch from 6fa0121 to a8cd502 Compare August 4, 2015 17:22
@ns-bot
Copy link

ns-bot commented Aug 4, 2015

@rosen-vladimirov rosen-vladimirov force-pushed the vladimirov/unify-proton-and-master branch from a8cd502 to 44349aa Compare August 5, 2015 06:49
@ns-bot
Copy link

ns-bot commented Aug 5, 2015

@rosen-vladimirov rosen-vladimirov force-pushed the vladimirov/unify-proton-and-master branch from 44349aa to 797ac57 Compare August 5, 2015 06:53
@ns-bot
Copy link

ns-bot commented Aug 5, 2015

As common lib is now a separate npm package, it has its own tests, that must be run when executing `grunt test` or `npm test` in nativescript-cli. Modify test command in package.json in order to run the tests.
Change Gruntfile.js in order to ignore/build the correct typescript files in common lib.
Add Bluebird as dependency as it is required for common lib.
Update mocha version.
Remove `chai` and `mocha` definitions as they are part of common lib now.
Update some dev dependencies.
@rosen-vladimirov rosen-vladimirov force-pushed the vladimirov/unify-proton-and-master branch from 797ac57 to 8ec9e09 Compare August 5, 2015 08:04
@ns-bot
Copy link

ns-bot commented Aug 5, 2015

@teobugslayer
Copy link
Contributor

👍

rosen-vladimirov added a commit that referenced this pull request Aug 5, 2015
…-master

Modify test infrastructure and grunt tasks
@rosen-vladimirov rosen-vladimirov merged commit 41ba966 into master Aug 5, 2015
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/unify-proton-and-master branch August 5, 2015 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants